Thanks a lot @val314159 !

@tmylk please review & prepare for merge.

It looks much better than #44
Though I'm missing some tests in this PR.

Tests from #44 need to be added here, then will be ready to merge. Should be a quick win.

Fixes #36 and #33

I'm not really sure how to test this exactly.... how do i fake out an entire ssh server?

Hi @val314159 , thank you for coming back to this!
For mocking one can use mock.patch as in issue 44

Ping @val314159, what's a status of this PR?

ping @val314159, what's a status of this PR?

I close because this PR is abandoned.

@menshikh-iv looks like only some tests are missing here -- can you add that and merge?

Need to finish this PR

What's missing:

• add paramiko dependency (are we fine with it @piskvorky, this is LGPL dependency)
• parametrize port (ssh not always on 22)
• add possibility to specify a key
• resolve merge conflict (this are really large)
• add tests (unit + integration)
  • check how it works with user/pass, ssh key auth, etc
  • how it works without username at all (only host, is this can retrieve a key from disk?)

anything else missing @mpenkov ?

Yes, linking against LGPL (without modifications) is fine.

This would come in handy.

Continued in #267